Skip to content

feat: Report open file descriptor count#58

Open
lfrancke wants to merge 1 commit intomainfrom
feat/fd-usage
Open

feat: Report open file descriptor count#58
lfrancke wants to merge 1 commit intomainfrom
feat/fd-usage

Conversation

@lfrancke
Copy link
Copy Markdown
Member

@lfrancke lfrancke commented Mar 30, 2026

Summary

  • Add open_files_count field to resources, read from /proc/self/fd - I checked but this seems to be the best way of getting that number. It seems a bit stupid to me counting this but....
  • Returns None gracefully if /proc/self/fd is unavailable even though I believe it should always be available

Test plan

  • cargo test --all-features passes
  • cargo clippy with RUSTFLAGS="-D warnings" passes

Read /proc/self/fd to report the actual number of open file
descriptors alongside the existing limit. Comparing usage vs limit
helps diagnose "too many open files" errors.
@lfrancke lfrancke self-assigned this Mar 30, 2026
@lfrancke lfrancke moved this to Development: Waiting for Review in Stackable Engineering Mar 30, 2026
Copy link
Copy Markdown
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, it looks silly, but I couldn't find a better alternative

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Waiting for Review

Development

Successfully merging this pull request may close these issues.

2 participants